Skip to content

Conversation

@TensorNull
Copy link
Contributor

- Add CometAPIChatModels type with 39+ models (GPT, Claude, Gemini, Grok, DeepSeek, Qwen)
- Add LLM.with_cometapi() static method for CometAPI support
- Base URL: https://api.cometapi.com/v1/
- API key from COMETAPI_KEY environment variable
- Tested with real API calls successfully
Copilot AI review requested due to automatic review settings October 14, 2025 03:51
@CLAassistant
Copy link

CLAassistant commented Oct 14, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds CometAPI integration to the OpenAI plugin, allowing users to access 500+ AI models from multiple providers through a unified API. The integration follows the same pattern as other provider integrations in the codebase.

  • Adds a new CometAPIChatModels type definition with models from GPT, Claude, Gemini, Grok, DeepSeek, and Qwen series
  • Implements a with_cometapi static method for creating CometAPI LLM instances
  • Includes comprehensive documentation and API key handling

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
models.py Defines the CometAPIChatModels literal type with supported model names
llm.py Adds import for the new model type and implements the with_cometapi static method

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@TensorNull
Copy link
Contributor Author

@tinalenguyen
Thank you for your careful review. I’ve made the corresponding changes. If there’s anything else you need me to do, please feel free to contact me anytime.

@tinalenguyen tinalenguyen merged commit d19519d into livekit:main Oct 21, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants